Description

I find that the numbered list is incorrectly indexed in two places.

Case 1: On Sub-list items

The indices of list items starting at level 2 to level 9 do not start over even though they are separate lists. Instead, they continue on the counter from the previous list. This is already mentioned at #6879.

Before

1. L1
  a. L2
    i. L3
2. L1
  b. L2 // This is supposed to be 'a.'
    ii.L3 // This is supposed to be 'i.'

After

1. L1
  a. L2
    i. L3
2. L1
  a. L2
    i.L3

Case 2: Created right after a bullet list

If a numbered list is created right after a bullet list, the indices continue from the previous list counter. It is caused by using the counter-reset instead of counter-set, making the reset step ineffective.

Before

1. L1
2. L1
Some random text
● Bullet list item 1
● Bullet list item 2
3. L1 // This is supposed to reset to '1'
4. L1 // This is supposed to '2'

After

1. L1
2. L1
Some random text
● Bullet list item 1
● Bullet list item 2
1. L1
2. L1

What to review

Overall, the counter is initialized under the .pt-editable element and will be reset to 0 at every element that is not a numbered list item. In addition, the counter increases by 1 at each numbered list item until the list ends. On the other hand, it resets the counter for elements not part of the numbered list.

In order to fix Case 1, I removed the code on line:98 and let the rule set for non-numbered list items take effect on bullet list items as well.

In order to fix Case 2, I decided to target the level 1 list item and reset the counter all levels between 2 - 9.

Testing

I test it manually by creating various lists which have multi-level nested items. In addition, I intend to add more different block elements between the lists to ensure the counter is correct.

Notes for release